Skip to content

Sort cached pipeline definition mapping by Def ID#129812

Open
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment
Open

Sort cached pipeline definition mapping by Def ID#129812
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment

Conversation

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Addresses the review feedback on #discussion_r3443995178 by applying only the requested documentation update: keep the cached mapping easy to scan by restoring Def ID ascending order.

  • Change made

    • Reordered rows in .github/skills/ci-pipeline-monitor/pipelines.md under Cached Definition ID Mapping so entries are sorted by numeric Def ID.
    • Kept the existing Private — skip rows at the end.
  • Example (before/after ordering intent)

    | runtime-coreclr libraries-jitstress2-jitstressregs | 119 | |
    | runtime-coreclr crossgen2 | 124 | |
    | runtime-coreclr crossgen2 outerloop | 134 | |
    | runtime-coreclr crossgen2-composite | 136 | |

Copilot AI requested review from Copilot and removed request for Copilot June 24, 2026 18:08
Co-authored-by: JulieLeeMSFT <63486087+JulieLeeMSFT@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 24, 2026 18:13
Copilot AI changed the title [WIP] Fix code based on review comment Sort cached pipeline definition mapping by Def ID Jun 24, 2026
Copilot AI requested a review from JulieLeeMSFT June 24, 2026 18:14
@github-actions github-actions Bot added the area-crossgen2-coreclr only use for closed issues label Jun 24, 2026
@JulieLeeMSFT JulieLeeMSFT added area-skills Agent Skills agentic-workflows and removed area-crossgen2-coreclr only use for closed issues labels Jun 25, 2026
@JulieLeeMSFT JulieLeeMSFT added this to the 11.0.0 milestone Jun 25, 2026
@JulieLeeMSFT JulieLeeMSFT marked this pull request as ready for review June 25, 2026 13:13
Copilot AI review requested due to automatic review settings June 25, 2026 13:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the CI pipeline monitor documentation by restoring ascending numeric order for the cached Azure DevOps pipeline definition IDs, keeping the mapping easy to scan while leaving “Private — skip” entries at the end.

Changes:

  • Reordered the “Cached Definition ID Mapping” table rows by ascending numeric Def ID.
  • Kept the “Private — skip” rows grouped at the end of the table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants